Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a newer version of ec in task definition #926

Merged

Conversation

simonbaird
Copy link
Contributor

@simonbaird simonbaird commented Apr 5, 2024

This should not be merged until after
EC-518 is resolved, since right now there are not v0.2 images of ec pushed out to registry.redhat.io. (Long story.)

It's expected this will resolve the root cause of
RHTAPBUGS-1212

I'm aware of the dire warning at the top of this file, but IIUC this PR to change ref does indeed belong here.

This should not be merged until *after*
https://issues.redhat.com/browse/EC-518 is resolved, since right now
there are not v0.2 images of ec pushed out to registry.redhat.io.
(Long story.)

It's expected this will resolve the root cause of
https://issues.redhat.com/browse/RHTAPBUGS-1212

I'm aware of the dire warning at the top of this file, but IIUC this
PR to change ref does indeed belong here.
@simonbaird
Copy link
Contributor Author

@joejstuart do you agree this PR does belong here?

Copy link

sonarqubecloud bot commented Apr 5, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@simonbaird
Copy link
Contributor Author

Draft until I can resolve EC-518.

@simonbaird
Copy link
Contributor Author

registry.redhat.io/rhtas-tech-preview/ec-rhel9:0.2 is live now, so this is ready for review and merge.

@simonbaird simonbaird marked this pull request as ready for review April 5, 2024 22:50
@zregvart zregvart merged commit 39ccf50 into konflux-ci:main Apr 7, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants